Skip to content

Skip failed jobs in benchmarking report#75

Merged
FloWuenne merged 6 commits intodevfrom
failed_job
Feb 26, 2025
Merged

Skip failed jobs in benchmarking report#75
FloWuenne merged 6 commits intodevfrom
failed_job

Conversation

@edmundmiller
Copy link
Contributor

@edmundmiller edmundmiller commented Feb 21, 2025

Follow up to #74

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@edmundmiller edmundmiller marked this pull request as ready for review February 25, 2025 17:58
nextflow.config Outdated
@@ -30,6 +30,7 @@ params {
// Benchmark report options
generate_benchmark_report = false
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
generate_benchmark_report = false
generate_benchmark_report = false

Copy link
Contributor

@FloWuenne FloWuenne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few changes. Mainly have to add the params.remove_failed_tasks to the proper location, which I couldn't add as a suggestion.

Copy link
Contributor

@FloWuenne FloWuenne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just incorporate changes and make sure to add params.remove_failed_tasks to proper location!

@FloWuenne FloWuenne added the enhancement New feature or request label Feb 25, 2025
@maxulysse
Copy link
Contributor

I think we can ignore the nfvalidation related tests as it's for the old nf-validation plugin, so It'll be dealt with in the TEMPLATE update

@FloWuenne FloWuenne merged commit b9becd4 into dev Feb 26, 2025
12 of 15 checks passed
@FloWuenne FloWuenne deleted the failed_job branch February 26, 2025 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants